Header/hero polish: fix + dog-ify the theme toggle, Forum link, image zoom#18
Conversation
- Fix the theme toggle: it was permanently hidden because theme.js set visibility='' (which clears the inline value and falls back to the CSS `visibility: hidden`) instead of 'visible'. Shipped broken in the dark-mode PR and never caught; add a test that the toggle is visible. - Nav: add a Forum link; point Discord at discord.apolloautomation.com. - Remove the footer blurb. - Hero: reword to the first-flash framing (devices update over Wi-Fi/OTA after setup, so the installer is for the first flash, not routine updates). - Hub card images: object-fit cover so cast-1 and msr-1 fill the card square like the square product shots (no effect on the already-square images). 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Duplicates the wiki's annotation-marker technique: paint the full-colour dog PNG directly (no CSS mask/tint) instead of the emoji. On the dark header the dog's dark outline blends in, so give it the same white backing it already has on the white light-mode header. The button keeps cycling system/dark/light, with the state carried by aria-label/title. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
|
Warning Review limit reached
Next review available in: 45 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (5)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
1 participant
What does this change?
Header/hero polish, a real bug fix for the theme toggle, and the Apollo dog mascot as the toggle icon.
theme.jsrevealed it withvisibility = '', which clears the inline value and falls back to the CSS.theme-btn { visibility: hidden }, so it never showed (shipped broken in the dark-mode PR; no test caught it). Now setsvisibility = 'visible', with a test asserting the toggle is visible.object-fit: coverso cast-1 and msr-1 fill the card square like the square product shots (the 8 already-square images are unaffected).Checklist
cd tests && npx playwright test) (20 passed, 1 intentional skip)cplist in.github/workflows/pages.yml(n/a: the dog PNG lives underassets/, which the cp-list already copies)devices.jsonchanged:python scripts/validate_registry.pypasses (devices.json unchanged)🤖 Generated with Claude Code